Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ThrownExceptionNotInFunction rule to the Analyzer Plugin #585

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

halotukozak
Copy link
Member

No description provided.

@halotukozak halotukozak requested a review from ddworak June 12, 2024 13:10
@halotukozak halotukozak self-assigned this Jun 12, 2024
@halotukozak
Copy link
Member Author

Closes #500

@halotukozak
Copy link
Member Author

It's a draft. We can consider wider solution to warn when Nothing provided in place of Function/PartialFunction argument.

@halotukozak halotukozak changed the title add ThrownExceptionNotInMonixScope rule to the Analyzer Plugin add ThrownExceptionNotInFunction rule to the Analyzer Plugin Jun 18, 2024
…. The Scala compiler loves to infer Nothing as a generic type but that is almost always incorrect. Explicit type arguments should be used instead.
…her type. The Scala compiler loves to infer Nothing as a generic type but that is almost always incorrect. Explicit type arguments should be used instead."

This reverts commit 9622a3f.
@halotukozak halotukozak force-pushed the 500-handle-thrown-exception-outside-the-monadic-context-of-task branch from fffed2f to ae2b07a Compare June 19, 2024 08:55
@halotukozak halotukozak force-pushed the 500-handle-thrown-exception-outside-the-monadic-context-of-task branch from 32644de to 0c3b54c Compare June 24, 2024 15:53
@halotukozak halotukozak requested a review from ddworak June 24, 2024 15:55
@halotukozak halotukozak linked an issue Jul 22, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handle thrown exception outside the monadic context of Task
2 participants